Skip to content

Fix parameter detection when using custom hql functions #2964

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
Jan 4, 2022
Merged

Fix parameter detection when using custom hql functions #2964

merged 8 commits into from
Jan 4, 2022

Conversation

maca88
Copy link
Contributor

@maca88 maca88 commented Dec 26, 2021

Fixes #2963

bahusoid
bahusoid reviewed Dec 26, 2021
View reviewed changes
@fredericDelaporte fredericDelaporte added this to the 5.3.11 milestone Dec 26, 2021
bahusoid
bahusoid reviewed Dec 27, 2021
View reviewed changes
@bahusoid bahusoid linked an issue Dec 27, 2021 that may be closed by this pull request
Time is incompatible with bigint for TimeAsTimeSpanType #2963
Closed
hazzik
hazzik reviewed Jan 1, 2022
View reviewed changes
hazzik
hazzik reviewed Jan 1, 2022
View reviewed changes
hazzik
hazzik reviewed Jan 1, 2022
View reviewed changes
hazzik
hazzik reviewed Jan 1, 2022
View reviewed changes
hazzik
hazzik reviewed Jan 1, 2022
View reviewed changes
@bahusoid bahusoid enabled auto-merge (squash) January 4, 2022 11:34
@bahusoid bahusoid disabled auto-merge January 4, 2022 11:35
@bahusoid bahusoid enabled auto-merge (squash) January 4, 2022 11:36
@bahusoid bahusoid merged commit 0d74ed5 into nhibernate:5.3.x Jan 4, 2022
@bahusoid bahusoid mentioned this pull request Jan 4, 2022
Closed
@fredericDelaporte fredericDelaporte mentioned this pull request Apr 18, 2022
Merged
@bahusoid bahusoid mentioned this pull request Mar 14, 2023
Merged
bahusoid added a commit that referenced this pull request Mar 16, 2023
@bahusoid
Cleanup guessed parameters handling in hql (#3255)
d419ab3 
Partial revert of no longer needed logic from #2964 (obsoleted by #2723)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hazzik hazzik hazzik approved these changes

@bahusoid bahusoid bahusoid approved these changes

Assignees

No one assigned

Labels

r: Fixed t: Fix

Projects

None yet

Milestone

5.3.11

Development

Successfully merging this pull request may close these issues.

Time is incompatible with bigint for TimeAsTimeSpanType

4 participants

@maca88 @hazzik @bahusoid @fredericDelaporte